chore: split heavy ML stack into optional mmory extras#42
Conversation
|
Thanks for this as well @AaineeSinha. Before merge, could you fix a couple small README things in the same PR?
CI should be fine as-is since it only installs |
|
Hi @rosspeili, I have updated the README.md to resolve the layout and reference issues. The code block closing fence has been fixed, the trailing text typo in the project structure tree has been removed, and a reference to issue #13 has been added to the long-term memory section. Ready for another look when you have a moment! |
|
Thanks for the update @AaineeSinha, project structure and the #13 mention look good. The optional install code block still needs a closing Once that’s in, good to merge from my side. Thanks again! |
|
Hi @rosspeili, I've closed the code block fence right after the optional memory install step so the ### 2. Configure defaults (optional) heading renders correctly now. |
Closes #35
This PR separates the heavy machine learning packages (
chromadb,sentence-transformers,torch) out of the mainrequirements.txtfile and moves them into a newrequirements-memory.txtfile.This ensures that the core CLI installation remains lightweight and fast for new users and CI environments while preserving an explicit installation path for future vector memory features.